Skip to content

feat(core): client tool and toolset loading with secure parameters - #779

Open
anubhav756 wants to merge 1 commit into
anubhav-secure-params-core-toolfrom
anubhav-secure-params-core-client
Open

feat(core): client tool and toolset loading with secure parameters#779
anubhav756 wants to merge 1 commit into
anubhav-secure-params-core-toolfrom
anubhav-secure-params-core-client

Conversation

@anubhav756

@anubhav756 anubhav756 commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Description

Adds secure_params support to ToolboxClient and ToolboxSyncClient for single-tool (load_tool) and batch toolset (load_toolset) discovery workflows.

Changes

  • Client load_tool accepts secure_params: Optional[Mapping[str, Any]] = None, validates parameter names against the discovered secure_parameters schema, and returns a pre-bound tool.
  • Client load_toolset accepts secure_params and supports strict=True/False validation across all tools in the loaded toolset via validate_unused_requirements().
  • Updated ToolboxSyncClient.load_tool() and load_toolset() to accept and forward secure_params.
  • Added unit tests covering load_tool and load_toolset with static values, dynamic callables, and strict validation errors.

@anubhav756
anubhav756 marked this pull request as ready for review August 20, 2026 06:13
@anubhav756
anubhav756 requested a review from a team as a code owner August 20, 2026 06:13
@anubhav756
anubhav756 force-pushed the anubhav-secure-params-core-client branch from 9e1ccaa to 3009e6b Compare August 20, 2026 06:25
@anubhav756
anubhav756 force-pushed the anubhav-secure-params-core-client branch from 3009e6b to 868d1d1 Compare August 20, 2026 10:08
@anubhav756
anubhav756 force-pushed the anubhav-secure-params-core-client branch from 868d1d1 to c41e1da Compare August 25, 2026 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants